Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.1] Add toolbar buttons in language installation toolbar #42610

Merged
merged 5 commits into from
Jan 16, 2024

Conversation

LadySolveig
Copy link
Contributor

Pull Request for Issue #35268 .

Summary of Changes

Added toolbar buttons to the language installation view to go directly to the language manage view or the content language view - just like in the extension installation view.

Testing Instructions

Go to System => Install => Languages

Actual result BEFORE applying this Pull Request

No toolbar buttons for manage languages and content languages

grafik

Expected result AFTER applying this Pull Request

A new toolbar button to go directly to manage languages and to content languages

grafik

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

* add toolbar button link to manage languages
* add toolbar button link content languages
* remove set title for toolbar - is already handled in the parent::addToolbar
* remove set title for toolbar - is already handled in the parent::addToolbar
* remove duplicate access request - is already handled in parent::addToolbar
* add language strings for toolbar buttons
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.1-dev labels Jan 4, 2024
* change order of language strings

Co-authored-by: Quy <[email protected]>
@brianteeman
Copy link
Contributor

Nothing against this PR but I don't see how it resolves #35268

@fgsw
Copy link

fgsw commented Jan 5, 2024

I have tested this item ✅ successfully on a76ae69

Thanks @LadySolveig


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42610.

@torstenhoppe
Copy link

I have tested this item ✅ successfully on 46cd2fc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42610.

@CCMLanger
Copy link

I have tested this item ✅ successfully on 46cd2fc

works fine


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42610.

@alikon
Copy link
Contributor

alikon commented Jan 8, 2024

Nothing against this PR but I don't see how it resolves #35268

a little step in the right direction

@alikon
Copy link
Contributor

alikon commented Jan 8, 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42610.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 8, 2024
@Razzo1987 Razzo1987 merged commit 97ff305 into joomla:5.1-dev Jan 16, 2024
3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 16, 2024
@Razzo1987
Copy link
Contributor

Thanks ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants